Skip to content

style: polish sidebar navigation active states and fix overflow scrol…#219

Merged
5 commits merged into
nensii21:mainfrom
itsaditi05:main
Jul 11, 2026
Merged

style: polish sidebar navigation active states and fix overflow scrol…#219
5 commits merged into
nensii21:mainfrom
itsaditi05:main

Conversation

@itsaditi05

Copy link
Copy Markdown

Description

This PR addresses the UI polish and alignment requirements mentioned in issue #165.

Changes Made

  • Enhanced the active navigation item state by introducing font-semibold for better visual distinction.
  • Added explicit keyboard focus rings (focus-visible:ring-2) to items for better accessibility.
  • Adjusted vertical padding to keep navigation links consistent with the profile footer link.
  • Fixed a layout glitch where a vertical browser scrollbar appeared inside the sidebar by hiding it using utility classes.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 Thank you for your first Pull Request to DevLink!

We appreciate your contribution to our open-source community.

Before your PR is reviewed, please ensure:

  • ✅ The project builds successfully.
  • ✅ Your code follows the project's coding standards.
  • ✅ You have tested your changes.
  • ✅ Related documentation has been updated if necessary.

Our maintainers will review your PR as soon as possible.

Thank you for helping improve DevLink! 💙

@nensii21

nensii21 commented Jul 9, 2026

Copy link
Copy Markdown
Owner

add screenshots and videos showing what changes have been made.Also CI job is failing please update it and let me know.

@itsaditi05

Copy link
Copy Markdown
Author

can u please approve the issue number #165.beacuse 8 workflows awaiting for approval

@nensii21 nensii21 closed this pull request by merging all changes into nensii21:main in 5bf96b8 Jul 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants